Closed Bug 1704855 Opened 4 years ago Closed 2 years ago

Make `--backgroundtask backgroundupdate` restart to apply update after staging

Categories

(Toolkit :: Application Update, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
118 Branch
Tracking Status
firefox118 --- fixed

People

(Reporter: nalexander, Assigned: nipunshukla002)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [fidedi-ope])

Attachments

(1 file)

Right now, --backgroundtask backgroundupdate will happily stage an update and exit. The next time it (or Firefox) runs, the staged update will be applied, completing the update. I'm not aware of a technical reason to do this in two steps. We elected to do so out of an abundance of caution: what happens if we get stuck in some kind of restart loop, constantly invoking the background task?

This ticket tracks doing better. We can try to restart the task after staging, or we can turn off staging in the background task and always apply, or maybe something else.

This doesn't block our initial release (i.e., milestone 1) -- it's just an improvement.

Priority: -- → P3
Whiteboard: [fidedi-ope]

I had this spontaneous idea, that maybe the task scheduler could also be used to schedule a preparation task to apply the update before the user starts Firefox again. I believe the scheduler can schedule tasks for the next idle state of the machine. There may be reasons against it and I already acknowledge, that it could make the testing more difficult, but it would probably reduce the felt startup times for some users.

Assignee: nobody → sshatrughan
Status: NEW → ASSIGNED
Status: ASSIGNED → NEW
Assignee: sshatrughan → nshukla
Status: NEW → ASSIGNED
Pushed by nshukla@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d82a8644ef7e Added automatic restart after staged --backgroundtask backgroundupdate r=nalexander,bytesized,application-update-reviewers
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: